feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/book/component/RecordItem.kt (2)

92-100: 감정 리소스 매핑 함수 public 전환: 안정성 주석 및 타입 힌트 추가 권장

• displayName(표시 문자열)에 의존하면 로케일/카피 변경 시 매핑이 깨질 수 있습니다. 가능하면 안정적인 코드/enum(예: 서버 코드나 core:model의 EmotionModel 등)에 기반한 매핑으로 전환을 권장합니다.
• 리소스 ID 반환에는 @DrawableRes 애노테이션을 부여해 호출 측 타입 안정성을 높여주세요.

예시:

+import androidx.annotation.DrawableRes
- fun getEmotionImageResourceByDisplayName(displayName: String): Int {
+ @DrawableRes
+ fun getEmotionImageResourceByDisplayName(displayName: String): Int {
   return when (displayName) {
     "따뜻함" -> R.drawable.img_warm
     "즐거움" -> R.drawable.img_joy
     "슬픔" -> R.drawable.img_sad
     "깨달음" -> R.drawable.img_insight
     else -> R.drawable.img_warm
   }
 }

추가 제안(선택): displayName 기반 함수는 유지하되, 내부적으로 안정 키→리소스 매핑을 우선하고, displayName→안정 키 변환 계층을 분리해두면 i18n 변화에 견고해집니다.

---

62-69: 빈 emotionTags 인덱싱으로 인한 충돌 가능성 (IndexOutOfBoundsException) 방지 필요

Preview에서 emotionTags = persistentListOf()로 비어있는 리스트를 넣고 있고, 런타임에서도 비어있을 수 있습니다. 현재 emotionTags[0] 접근은 즉시 크래시를 유발합니다.

안전 접근으로 변경 제안:

- Image(
-     painter = painterResource(getEmotionImageResourceByDisplayName(emotionTags[0])),
-     contentDescription = "Emotion Graphic",
-     modifier = Modifier
-         .size(40.dp)
-         .clip(CircleShape),
- )
+ val primaryEmotion = emotionTags.firstOrNull()
+ if (primaryEmotion != null) {
+   Image(
+     painter = painterResource(getEmotionImageResourceByDisplayName(primaryEmotion)),
+     contentDescription = stringResource(R.string.cd_emotion_graphic),
+     modifier = Modifier
+       .size(40.dp)
+       .clip(CircleShape),
+   )
+ }

태그 텍스트도 동일하게 보호:

- Text(
-   text = "#${emotionTags[0]}",
-   ...
- )
+ if (primaryEmotion != null) {
+   Text(
+     text = "#$primaryEmotion",
+     ...
+   )
+ }

참고: R.string.cd_emotion_graphic는 strings.xml에 추가 필요합니다(아래 i18n 코멘트 참고).

Also applies to: 70-76

core/common/src/main/kotlin/com/ninecraft/booket/core/common/constants/ErrorScope.kt (1)

3-5: KDoc 보강 제안 (의도와 사용처 명시)

간단한 주석만으로도 가독성과 일관성이 높아집니다. 아래처럼 추가를 고려해주세요.

 package com.ninecraft.booket.core.common.constants

+/**
+ * 에러가 발생/표시되는 영역을 구분합니다.
+ * - GLOBAL: 앱 전역(메인 등)에서의 에러
+ * - LOGIN: 로그인/세션 관련 에러
+ * - BOOK_REGISTER: 책 등록 플로우 관련 에러
+ * - RECORD_REGISTER: 기록 등록 플로우 관련 에러
+ */
 enum class ErrorScope {
-    GLOBAL, LOGIN, BOOK_REGISTER, RECORD_REGISTER
+    GLOBAL, LOGIN, BOOK_REGISTER, RECORD_REGISTER
 }

core/ui/src/main/res/values/strings.xml (1)

5-6: 문구/띄어쓰기 미세 개선 제안 (선택사항)

사내 문체 가이드에 따라 아래 형태도 고려해 보세요.

-<string name="network_error_message">네트워크 연결이 불안정합니다.\n인터넷 연결을 확인해주세요</string>
-<string name="server_error_message">알 수 없는 문제가 발생했어요.\n다시 시도해주세요</string>
+<string name="network_error_message">네트워크 연결이 불안정합니다.\n인터넷 연결을 확인해 주세요</string>
+<string name="server_error_message">알 수 없는 문제가 발생했어요.\n잠시 후 다시 시도해 주세요</string>

또한, 다이얼로그/풀스크린 UI에서 줄바꿈 적용이 일관적으로 보이는지 실제 기기에서 한 번만 더 확인해 주세요.

feature/home/src/main/kotlin/com/ninecraft/booket/feature/home/HomeUiState.kt (1)

12-17: @immutable 주석 추가 고려

Compose 재구성 비용 감소 및 분석기 힌트를 위해 UiState에도 @immutable 적용을 권장합니다.

 sealed interface UiState {
+    @Immutable
+    // 또는 sealed interface 자체에 @Immutable 부여
     data object Idle : UiState
     data object Loading : UiState
     data object Success : UiState
     data class Error(val exception: Throwable) : UiState
 }

참고: sealed interface 자체에 @immutable를 붙이는 방식도 가능합니다(팀 컨벤션에 맞춰 선택).

feature/library/src/main/kotlin/com/ninecraft/booket/feature/library/LibraryUiState.kt (1)

13-18: @immutable 적용으로 Compose 친화성 강화

Home과 동일하게 UiState에도 @immutable 부여를 권장합니다.

 sealed interface UiState {
+    @Immutable
     data object Idle : UiState
     data object Loading : UiState
     data object Success : UiState
     data class Error(val exception: Throwable) : UiState
 }

feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/book/component/RecordItem.kt (1)

64-64: contentDescription 하드코딩(i18n) → stringResource 사용 권장

접근성 문자열은 현지화 리소스로 관리해야 합니다.

예시:

- contentDescription = "Emotion Graphic",
+ contentDescription = stringResource(R.string.cd_emotion_graphic),

필요 시 feature/detail 모듈의 res/values/strings.xml에 cd_emotion_graphic 키를 추가해 주세요. 매개값(예: 감정명)을 포함시키려면 포맷 문자열(%1$s)로 정의 후 stringResource(R.string.cd_emotion_graphic, primaryEmotion) 형태로 사용 가능합니다.

feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/record/component/ReviewBox.kt (1)

24-24: 교차 컴포넌트 의존 최소화를 위한 유틸 분리 제안

getEmotionImageResourceByDisplayName가 ‘book/component/RecordItem.kt’에 위치한 채 ‘record/component’에서 참조되고 있습니다. detail 기능 전반에서 사용하는 매핑이므로 feature/detail/.../common/EmotionResources.kt 같은 파일로 추출해 두면 모듈(패키지) 간 결합이 낮아지고 재사용/테스트가 수월해집니다.

feature/library/src/main/kotlin/com/ninecraft/booket/feature/library/LibraryPresenter.kt (1)

90-97: 에러 표현 형식 통일 제안

UiState에는 Throwable을 그대로 넘기는데, 같은 지점의 footerState는 문자열만 넘기고 있습니다.
두 컴포넌트가 다른 타입을 요구하더라도, 한쪽만 예외 객체·한쪽만 메시지로 관리하면 추후 파싱/로깅 로직이 이원화됩니다.
가능하다면 FooterState.Error도 예외 객체를 받아 내부에서 메시지를 파생하도록 통일해보세요.

core/common/src/main/kotlin/com/ninecraft/booket/core/common/constants/ErrorDialogSpec.kt (1)

3-7: action 기본값 추가 제안

대부분의 에러 다이얼로그는 별도 액션 없이 닫기만 합니다. action: () -> Unit = {} 로 기본값을 주면 호출부에서 람다 생략이 가능해 코드가 간결해집니다.

core/ui/src/main/kotlin/com/ninecraft/booket/core/ui/component/ReedDialog.kt (1)

66-73: 불필요한 변수 참조

title?.let { … Text(text = title, …) } 구문에서 람다 내부는 it 를 바로 쓰는 편이 명확합니다.

- title?.let {
-     Text(
-         text = title,
+ title?.let {
+     Text(
+         text = it,

feature/main/src/main/kotlin/com/ninecraft/booket/feature/main/MainActivity.kt (2)

59-69: 전역 에러 이벤트 연속 수신 시 덮어쓰기 가능성

다이얼로그가 떠있는 동안 새 이벤트가 오면 기존 스펙을 덮어씁니다. UX 상 혼란이 있을 수 있으니 큐잉 또는 표시 중엔 드롭 등 정책을 정해 주세요. 간단히 드롭하려면 아래처럼 가드 가능합니다.

 LaunchedEffect(Unit) {
     ErrorEventHelper.errorEvent.collect { event ->
-        when (event) {
+        if (dialogSpec.value != null) return@collect
+        when (event) {
             is ErrorEvent.ShowDialog -> {
                 dialogSpec.value = event.spec
             }
         }
     }
 }

필요 시 SnapshotStateList로 큐를 두고, confirm/dismiss 시 다음 항목을 표시하는 방식도 고려해 보세요.

---

57-57: 설정 변경 시 다이얼로그 유지가 필요하다면 saveable 고려

dialogSpec는 remember 상태라 회전 등 구성 변경 시 사라집니다. 유지가 필요하면 Saver를 제공해 rememberSaveable로 전환을 검토해 주세요(단, ErrorDialogSpec이 직렬화 가능해야 합니다).

core/ui/src/main/kotlin/com/ninecraft/booket/core/ui/component/ReedErrorUi.kt (2)

23-31: 재사용성 향상을 위한 modifier 파라미터 추가 제안

항상 fillMaxSize로 고정되어 있어 섹션 단위(부분 영역) 에러 UI 재사용이 어렵습니다. modifier를 받아 확장 가능하게 해주세요.

-@Composable
-fun ReedErrorUi(
-    exception: Throwable,
-    onRetryClick: () -> Unit,
-) {
+@Composable
+fun ReedErrorUi(
+    exception: Throwable,
+    onRetryClick: () -> Unit,
+    modifier: Modifier = Modifier,
+) {
-    val message = if (exception.isNetworkError()) stringResource(R.string.network_error_message) else stringResource(R.string.server_error_message)
-    Box(
-        modifier = Modifier.fillMaxSize(),
+    val message =
+        if (exception.isNetworkError())
+            stringResource(R.string.network_error_message)
+        else
+            stringResource(R.string.server_error_message)
+    Box(
+        modifier = modifier.fillMaxSize(),
         contentAlignment = Alignment.Center,
     ) {

---

27-27: 가독성: 메시지 분기 한 줄 표기 → 여러 줄로 정리(선택)

조건식이 길어 가독성이 떨어집니다. when/여러 줄 if로 정리하면 읽기 쉬워집니다.

-    val message = if (exception.isNetworkError()) stringResource(R.string.network_error_message) else stringResource(R.string.server_error_message)
+    val message = if (exception.isNetworkError()) {
+        stringResource(R.string.network_error_message)
+    } else {
+        stringResource(R.string.server_error_message)
+    }

core/common/src/main/kotlin/com/ninecraft/booket/core/common/utils/HandleException.kt (1)

65-93: 하드코딩된 오류 메시지를 문자열 리소스로 분리 권장

buildDialog 내부의 한국어 메시지가 코드에 직접 하드코딩돼 있습니다.
향후 다국어 지원 및 유지보수를 위해 core/common/res/values/strings.xml 등의 리소스로 옮겨 주세요.
이 변경은 UI 모듈의 문자열 관리 방식과도 일관성을 맞추는 데 도움이 됩니다.

feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/book/BookDetailUi.kt (4)

136-145: Idle 상태에서 빈 화면 대신 Loading과 통합해 UX 플리커를 줄여주세요.

초기 진입 시 Idle → 즉시 Loading 전환되는 동안 빈 화면이 잠깐 보일 수 있습니다. Idle과 Loading을 묶어 동일한 로딩 UI를 보여주는 편이 자연스럽습니다.

-    when (state.uiState) {
-        is UiState.Idle -> {}
-        is UiState.Loading -> {
+    when (state.uiState) {
+        is UiState.Idle,
+        is UiState.Loading -> {
             Box(
                 modifier = Modifier.fillMaxSize(),
                 contentAlignment = Alignment.Center,
             ) {
                 CircularProgressIndicator(color = ReedTheme.colors.contentBrand)
             }
         }

---

187-192: 접근성: 장식 아이콘은 contentDescription을 null로 설정

해당 아이콘은 드롭다운 표시용 장식 요소로 보입니다. 버튼 텍스트가 의미를 전달하므로 스크린리더에 노출하지 않도록 contentDescription = null을 권장합니다.

-                                    Icon(
-                                        imageVector = ImageVector.vectorResource(designR.drawable.ic_chevron_down),
-                                        contentDescription = "Dropdown Icon",
+                                    Icon(
+                                        imageVector = ImageVector.vectorResource(designR.drawable.ic_chevron_down),
+                                        contentDescription = null,
                                         modifier = Modifier.size(22.dp),
                                         tint = ReedTheme.colors.contentPrimary,
                                     )

---

253-273: Lazy 리스트는 items(list) 변형을 사용하면 더 간결하고 안전합니다

인덱스 접근 기반 items(count) 대신 items(items, key)가 가독성/안정성(리스트 변경 시 OOB 방지)에 유리합니다.

-                } else {
-                    items(
-                        count = state.readingRecords.size,
-                        key = { index -> state.readingRecords[index].id },
-                    ) { index ->
-                        val record = state.readingRecords[index]
+                } else {
+                    items(
+                        items = state.readingRecords,
+                        key = { record -> record.id },
+                    ) { record ->
                         RecordItem(
                             quote = record.quote,
                             emotionTags = record.emotionTags.toImmutableList(),
                             pageNumber = record.pageNumber,
                             createdAt = record.createdAt.toFormattedDate(),
                             modifier = Modifier
                                 .padding(
                                     start = ReedTheme.spacing.spacing5,
                                     end = ReedTheme.spacing.spacing5,
                                     bottom = ReedTheme.spacing.spacing3,
                                 )
                                 .clickable {
-                                    state.eventSink(BookDetailUiEvent.OnRecordItemClick(record.id))
+                                    state.eventSink(BookDetailUiEvent.OnRecordItemClick(record.id))
                                 },
                         )
                     }

---

301-303: Preview 데이터 충분성 검증 요청

UiState.Success만 지정되어 있는데, BookDetailUiState의 다른 필드(예: bookDetail, readingRecords, seedsStats 등)가 non-null/기본값을 요구한다면 프리뷰가 깨질 수 있습니다. 팀에서 더미 데이터를 선호하므로, preview 전용 팩토리/스텁(state.preview())를 제공하거나 프리뷰 내에서 최소 더미를 세팅하는 것을 권장합니다.

원하시면 preview용 스텁 생성/적용 패치를 작성해 드리겠습니다.

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#46
File: feature/search/src/main/kotlin/com/ninecraft/booket/feature/search/component/InfiniteLazyColumn.kt:83-95
Timestamp: 2025-07-14T00:46:03.843Z
Learning: seoyoon513과 팀은 한국어 주석을 선호하며, 한국어 주석을 영어로 번역하라는 제안을 하지 않아야 함

Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#75
File: feature/home/src/main/kotlin/com/ninecraft/booket/feature/home/HomeUi.kt:0-0
Timestamp: 2025-07-29T06:07:11.727Z
Learning: seoyoon513 팀에서는 UI 구현 단계에서 더미 데이터를 하드코딩하여 화면을 먼저 구현하고, 이후 서버 연동 시점에 실제 데이터로 교체하는 개발 방식을 사용합니다.

Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#72
File: feature/record/src/main/kotlin/com/ninecraft/booket/feature/record/ocr/OcrPresenter.kt:70-70
Timestamp: 2025-07-28T18:22:00.618Z
Learning: LiveTextAnalyzer는 ML Kit 에러를 addOnFailureListener로 처리하지만 로그만 남기고 presenter에 실패를 알리지 않아서, UI에서 텍스트 인식 실패 상태를 표시할 수 없는 문제가 있음. StillTextAnalyzer처럼 실패 콜백을 추가하는 것이 더 나은 해결책임

Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#88
File: core/model/src/main/kotlin/com/ninecraft/booket/core/model/EmotionModel.kt:11-18
Timestamp: 2025-07-31T16:58:59.404Z
Learning: Reed-Android 프로젝트는 현재 다국어 지원 계획이 없어서 모델에 한글 문자열을 직접 포함하는 것이 허용된다.

Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#88
File: core/model/src/main/kotlin/com/ninecraft/booket/core/model/EmotionModel.kt:11-18
Timestamp: 2025-07-31T16:58:59.404Z
Learning: Reed-Android 프로젝트에서 core:model 모듈은 순수 Kotlin 모듈이므로 Android 리소스(R.string 등)에 접근할 수 없다.

Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#88
File: feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/book/component/RecordItem.kt:29-37
Timestamp: 2025-07-31T23:17:40.054Z
Learning: Reed-Android 프로젝트에서는 API가 준비되지 않은 상황에서 UI를 먼저 구현하고, API 연동 시점에 하드코딩된 데이터를 실제 데이터로 교체하는 개발 방식을 사용한다. RecordItem 컴포넌트의 emotionTags 매개변수도 API 연동 시점에 `text = emotionTags.joinToString(separator = "·") { "#$it" }`로 적용될 예정이다.

Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#61
File: feature/webview/build.gradle.kts:17-21
Timestamp: 2025-07-20T12:34:23.786Z
Learning: Reed-Android 프로젝트에서는 `booket.android.feature` convention plugin을 사용하여 feature 모듈들의 공통 의존성을 관리한다. 이 plugin은 Circuit, Compose, 그리고 core 모듈들의 의존성을 자동으로 포함하므로, 각 feature 모듈의 build.gradle.kts에서는 특별한 의존성(예: libs.logger, libs.kakao.auth)만 별도로 선언하면 된다.

Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#88
File: feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/book/component/RecordsCollection.kt:25-59
Timestamp: 2025-07-31T23:22:02.816Z
Learning: BookDetailScreen의 RecordsCollection 컴포넌트는 상위 Column의 verticalScroll과 LazyColumn의 무한 스크롤을 동시에 지원해야 하는 중첩 스크롤 시나리오입니다. LazyColumn에 고정 높이를 설정하지 않으면 앱이 충돌하는 이슈가 있어서, 현재는 높이를 계산하여 설정하고 userScrollEnabled = false로 구현되어 있습니다. 향후 InfiniteLazyColumn과 nestedScrollConnection을 도입하여 전체 화면 스크롤 + LazyColumn 무한 스크롤을 지원할 예정입니다.

Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#35
File: feature/login/src/main/kotlin/com/ninecraft/booket/feature/login/TermsAgreementScreen.kt:127-127
Timestamp: 2025-07-09T01:14:29.836Z
Learning: In the Reed-Android project's TermsAgreementScreen.kt, the OnTermDetailClick event is intentionally passed an empty string for the URL parameter because the actual URLs for terms detail pages haven't been decided yet. This is a temporary implementation that will be updated once the URLs are finalized.

Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#72
File: feature/record/src/main/kotlin/com/ninecraft/booket/feature/record/ocr/component/CameraFrame.kt:21-82
Timestamp: 2025-07-28T18:08:47.298Z
Learning: Jetpack Compose에서 scale() 변환은 시각적 변환만 적용하며 레이아웃 좌표계는 변경하지 않는다. 따라서 scale(scaleX = -1f, scaleY = -1f)로 반전된 아이콘에서 padding()은 원래 레이아웃 기준으로 동작하므로, 시각적으로 올바른 위치를 위해서는 변환 전 좌표계 기준으로 padding을 설정해야 한다.

Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#45
File: core/designsystem/src/main/kotlin/com/ninecraft/booket/core/designsystem/component/appbar/ReedTopAppBar.kt:65-65
Timestamp: 2025-07-12T01:33:57.101Z
Learning: Reed Android 프로젝트에서 타이포그래피 사용 규칙: 톱 앱바(Top App Bar)에서는 `headline2SemiBold`를 사용하고, 바텀시트(Bottom Sheet)에서는 `heading2SemiBold`를 사용한다. 이는 의도적인 디자인 시스템 차별화이다.

Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#77
File: feature/onboarding/src/main/kotlin/com/ninecraft/booket/feature/onboarding/OnboardingUi.kt:73-73
Timestamp: 2025-07-29T07:02:18.885Z
Learning: Kotlin에서 같은 패키지 내의 파일들은 패키지 레벨에 정의된 const val 상수를 import 없이 직접 접근할 수 있습니다. OnboardingPresenter.kt와 OnboardingUi.kt 모두 com.ninecraft.booket.feature.onboarding 패키지에 속해 있어서 ONBOARDING_STEPS_COUNT 상수를 공유할 수 있습니다.

Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#52
File: feature/main/src/main/kotlin/com/ninecraft/booket/feature/main/screens/DelegatingNavigator.kt:0-0
Timestamp: 2025-07-16T15:54:19.322Z
Learning: In the Reed-Android project using Circuit architecture, DelegatingNavigator was restored because NavigableCircuitContent can only inject a single navigator, but the dual-navigator architecture (childNavigator for bottom navigation screens, rootNavigator for full-screen screens) requires a delegating component to work within Circuit's constraint.

Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#88
File: feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/record/RecordDetailPresenter.kt:18-47
Timestamp: 2025-07-31T23:30:37.547Z
Learning: In Circuit architecture, presenters receive the Screen object directly as a constructor parameter (e.g., Assisted private val screen: RecordDetailScreen), and screen parameters are accessed through this screen object (e.g., screen.recordId). Screen parameters should not be added as separate constructor parameters.

Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#52
File: feature/main/src/main/kotlin/com/ninecraft/booket/feature/main/screens/DelegateNavigator.kt:0-0
Timestamp: 2025-07-16T16:11:27.044Z
Learning: In the Reed-Android project's DelegateNavigator implementation, the pop(), peek(), and peekBackStack() methods should always use childNavigator without branching logic, as they operate on the currently active navigation stack. Only goTo() and resetRoot() methods need to route between childNavigator and rootNavigator based on screen type.

Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#52
File: feature/main/src/main/kotlin/com/ninecraft/booket/feature/main/screens/DelegateNavigator.kt:0-0
Timestamp: 2025-07-16T16:11:27.044Z
Learning: In the Reed-Android project's DelegateNavigator implementation, the pop(), peek(), and peekBackStack() methods should always use childNavigator without branching logic, as they operate on the currently active navigation stack. Only goTo() and resetRoot() methods need to route between childNavigator and rootNavigator based on screen type using isMainTabScreen() function.

Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#32
File: core/designsystem/src/main/kotlin/com/ninecraft/booket/core/designsystem/component/button/ButtonColorStyle.kt:10-16
Timestamp: 2025-07-08T12:33:01.863Z
Learning: Reed Android 프로젝트에서 KAKAO 버튼 스타일은 디자이너가 pressed 상태 색상을 별도로 정의하지 않았기 때문에 pressed 상태에서도 동일한 Kakao 색상을 사용한다.

Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#63
File: feature/library/src/main/kotlin/com/ninecraft/booket/feature/library/LibraryPresenter.kt:28-53
Timestamp: 2025-07-22T05:19:10.071Z
Learning: feature/library의 LibraryPresenter에서 현재 FilterChipState의 카운트 값들은 UI 확인용 더미 데이터이며, API가 준비되면 실제 데이터로 교체될 예정입니다.

Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#46
File: core/network/src/main/kotlin/com/ninecraft/booket/core/network/response/BookSearchResponse.kt:33-44
Timestamp: 2025-07-14T00:51:38.952Z
Learning: The project uses Kotlinx Serialization with JSON configuration that includes `ignoreUnknownKeys = true`, `encodeDefaults = true`, `isLenient = true`, and `prettyPrint = true` to handle API response parsing robustly, making additional null safety measures less critical if the API consistently provides all expected fields.

feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/book/BookDetailPresenter.kt (1)

97-101: 첫 페이지 로드 후 isLastPage·currentStartIndex 갱신 누락
초기 로드가 완료돼도 두 변수가 초기값(0/false)에 머물러 PAGE_SIZE 미만 데이터일 때 추가 로드를 계속 시도합니다.
지난 리뷰에서도 동일 지적이 있었으므로 반드시 반영해 주세요.

bookDetail = detail
seedsStates = seeds.categories.toImmutableList()
readingRecords = records.content.toPersistentList()

currentStartIndex = START_INDEX
isLastPage = records.content.size < PAGE_SIZE

feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/book/BookDetailUi.kt (2)

135-144: 로딩 인디케이터 접근성 개선 필요

CircularProgressIndicator에 contentDescription이 없어 스크린리더 사용 시 로딩 상태를 인지할 수 없습니다.
Modifier.semantics { contentDescription = ... } 또는 progressSemantics()를 적용해 주세요.

---

284-289: Error UI 크기 지정 누락

ReedErrorUi가 Modifier.fillMaxSize() 없이 배치되면 상위 레이아웃에 따라 중앙 정렬이 깨질 수 있습니다.
modifier = Modifier.fillMaxSize()를 넘겨 전체 화면을 일관되게 덮도록 하는 편이 안전합니다.

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#46
File: feature/search/src/main/kotlin/com/ninecraft/booket/feature/search/component/InfiniteLazyColumn.kt:83-95
Timestamp: 2025-07-14T00:46:03.843Z
Learning: seoyoon513과 팀은 한국어 주석을 선호하며, 한국어 주석을 영어로 번역하라는 제안을 하지 않아야 함

Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#75
File: feature/home/src/main/kotlin/com/ninecraft/booket/feature/home/HomeUi.kt:0-0
Timestamp: 2025-07-29T06:07:11.727Z
Learning: seoyoon513 팀에서는 UI 구현 단계에서 더미 데이터를 하드코딩하여 화면을 먼저 구현하고, 이후 서버 연동 시점에 실제 데이터로 교체하는 개발 방식을 사용합니다.

Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#61
File: feature/webview/build.gradle.kts:17-21
Timestamp: 2025-07-20T12:34:23.786Z
Learning: Reed-Android 프로젝트에서는 `booket.android.feature` convention plugin을 사용하여 feature 모듈들의 공통 의존성을 관리한다. 이 plugin은 Circuit, Compose, 그리고 core 모듈들의 의존성을 자동으로 포함하므로, 각 feature 모듈의 build.gradle.kts에서는 특별한 의존성(예: libs.logger, libs.kakao.auth)만 별도로 선언하면 된다.

Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#88
File: feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/book/component/RecordsCollection.kt:25-59
Timestamp: 2025-07-31T23:22:02.816Z
Learning: BookDetailScreen의 RecordsCollection 컴포넌트는 상위 Column의 verticalScroll과 LazyColumn의 무한 스크롤을 동시에 지원해야 하는 중첩 스크롤 시나리오입니다. LazyColumn에 고정 높이를 설정하지 않으면 앱이 충돌하는 이슈가 있어서, 현재는 높이를 계산하여 설정하고 userScrollEnabled = false로 구현되어 있습니다. 향후 InfiniteLazyColumn과 nestedScrollConnection을 도입하여 전체 화면 스크롤 + LazyColumn 무한 스크롤을 지원할 예정입니다.

Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#77
File: feature/onboarding/src/main/kotlin/com/ninecraft/booket/feature/onboarding/OnboardingUi.kt:73-73
Timestamp: 2025-07-29T07:02:18.885Z
Learning: Kotlin에서 같은 패키지 내의 파일들은 패키지 레벨에 정의된 const val 상수를 import 없이 직접 접근할 수 있습니다. OnboardingPresenter.kt와 OnboardingUi.kt 모두 com.ninecraft.booket.feature.onboarding 패키지에 속해 있어서 ONBOARDING_STEPS_COUNT 상수를 공유할 수 있습니다.

Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#52
File: feature/main/src/main/kotlin/com/ninecraft/booket/feature/main/screens/DelegatingNavigator.kt:0-0
Timestamp: 2025-07-16T15:54:19.322Z
Learning: In the Reed-Android project using Circuit architecture, DelegatingNavigator was restored because NavigableCircuitContent can only inject a single navigator, but the dual-navigator architecture (childNavigator for bottom navigation screens, rootNavigator for full-screen screens) requires a delegating component to work within Circuit's constraint.

feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/book/BookDetailPresenter.kt (1)

241-244: 재시도 시 페이징 상태 초기화 누락 — 이전 리뷰와 동일

initialLoad() 호출 전에 footerState(특히 Error 상태) 를 Idle 로,
readingRecords, isLastPage, currentStartIndex 등을 초기화하지 않으면
재시도 후에도 잘못된 페이징 UI 가 남을 수 있습니다.

이 부분은 이전 커밋에서도 지적되었으므로 참고 바랍니다.

feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/book/BookDetailPresenter.kt (2)

82-95: 동시 요청 실패 시 다른 요청까지 취소될 수 있습니다

coroutineScope { async { … } } 구조에서는 하나의 네트워크 호출이 예외를 던지면 나머지 async 작업도 전파된 취소로 중단됩니다.
각 요청을 독립적으로 시도하고, 일부만 실패해도 전체 UI 를 에러로 전환할지 선택하려면 supervisorScope 로 감싸는 편이 안전합니다.

supervisorScope {
    val detailDef = async { … }
    val seedsDef  = async { … }
    val recordsDef = async { … }
    // 필요에 따라 개별 await + 개별 에러 처리
}

이렇게 하면 특정 호출 실패 시 다른 호출이 불필요하게 취소되는 것을 방지할 수 있습니다.

---

135-139: 다이얼로그와 토스트가 중복으로 표시됩니다

postErrorDialog() 로 오류 다이얼로그를 띄운 뒤 바로 아래에서 동일 오류를 ShowToast 로도 알리고 있습니다.
사용자에게 동일 메시지가 두 번 노출되어 UX 가 과도할 수 있으니 하나만 남기고 통일하는 것을 권장드립니다.

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#46
File: feature/search/src/main/kotlin/com/ninecraft/booket/feature/search/component/InfiniteLazyColumn.kt:83-95
Timestamp: 2025-07-14T00:46:03.843Z
Learning: seoyoon513과 팀은 한국어 주석을 선호하며, 한국어 주석을 영어로 번역하라는 제안을 하지 않아야 함

Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#75
File: feature/home/src/main/kotlin/com/ninecraft/booket/feature/home/HomeUi.kt:0-0
Timestamp: 2025-07-29T06:07:11.727Z
Learning: seoyoon513 팀에서는 UI 구현 단계에서 더미 데이터를 하드코딩하여 화면을 먼저 구현하고, 이후 서버 연동 시점에 실제 데이터로 교체하는 개발 방식을 사용합니다.

Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#88
File: feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/book/component/RecordsCollection.kt:25-59
Timestamp: 2025-07-31T23:22:02.816Z
Learning: BookDetailScreen의 RecordsCollection 컴포넌트는 상위 Column의 verticalScroll과 LazyColumn의 무한 스크롤을 동시에 지원해야 하는 중첩 스크롤 시나리오입니다. LazyColumn에 고정 높이를 설정하지 않으면 앱이 충돌하는 이슈가 있어서, 현재는 높이를 계산하여 설정하고 userScrollEnabled = false로 구현되어 있습니다. 향후 InfiniteLazyColumn과 nestedScrollConnection을 도입하여 전체 화면 스크롤 + LazyColumn 무한 스크롤을 지원할 예정입니다.